Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New extract-queries command + other improvements #115

Merged
merged 5 commits into from
Jan 23, 2025
Merged

Conversation

hannahbast
Copy link
Member

@hannahbast hannahbast commented Jan 14, 2025

The new extract-queries command extracts all queries from the server log and writes it into a TSV file that can be processed using the example-queries command. If the server log contains lines matching Alive check with message "...", the message is used as base name for the query descriptions. For each query with that base name Query #<index> is appended to the description.

In this context, the following small improvements for the example-queries command (items 1, 2, 3, 4) and the settings command (item 5) were made:

  1. The default for --download_or_count is now the more natural download (was: count).
  2. The option --accept now has two more choices application/qlever-results+json and AUTO, where the later picks the accept header automatically from the query type
  3. There is a new option --add-query-type-to-description
  4. Long query descriptions are now truncated in the middle, that is shown as Beginnig ... end
  5. For the runtime parameters supported by the settings command, there is now also always-mulitply-unions

Hannah Bast added 5 commits January 14, 2025 05:34
Extracts queries from query log

TODO: The `example-queries` command currently does not work properly
with a mix of SELECT and CONSTRUCT queries because they require
differnt Accept headers
@hannahbast hannahbast changed the title New extract-queries command New extract-queries command + other improvements Jan 23, 2025
@hannahbast hannahbast merged commit 94c0f79 into main Jan 23, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant